Skip to content

feat: Migrate package manager to uv#1553

Merged
Naramsim merged 7 commits into
PokeAPI:masterfrom
FallenDeity:feat/migrate-to-uv
Jul 4, 2026
Merged

feat: Migrate package manager to uv#1553
Naramsim merged 7 commits into
PokeAPI:masterfrom
FallenDeity:feat/migrate-to-uv

Conversation

@FallenDeity

@FallenDeity FallenDeity commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Change description

First set of pr for migrating and adopting modern python tooling ecosystem, this pr attempts to migrate from pip to uv for package management as discussed in #1391 & #1521 it breaks down the changes @phalt initially made to smaller more iterative parts

cc: @phalt @Naramsim

AI coding assistance disclosure

Used a bit for reviewing the changes and coverage if I missed any transitive command update

Contributor check list

  • I have written a description of the contribution and explained its motivation.
  • I have written tests for my code changes (if applicable).
  • I have read and understood the AI Assisted Contribution guidelines.
  • I will own this change in production, and I am prepared to fix any bugs caused by my code change.

@FallenDeity FallenDeity marked this pull request as ready for review June 10, 2026 19:19
Naramsim
Naramsim reviewed Jun 11, 2026
View reviewed changes
Comment thread .circleci/config.yml
Comment thread .python-version
Comment thread docker-compose.yml
jemarq04
jemarq04 reviewed Jun 16, 2026
View reviewed changes
Comment thread .github/workflows/database.yml Outdated
Comment thread Resources/docker/app/Dockerfile Outdated
Comment thread .python-version
Comment thread Makefile
@FallenDeity FallenDeity requested review from Naramsim and jemarq04 June 17, 2026 15:58
@Naramsim

Naramsim commented Jun 24, 2026

Copy link
Copy Markdown
Member

I want to review this PR, don't merge please. I'm slow I know, bear with me

@Naramsim Naramsim mentioned this pull request Jun 26, 2026
Draft
Naramsim
Naramsim requested changes Jun 26, 2026
View reviewed changes
Comment thread Makefile Outdated
Comment thread pyproject.toml Outdated
Comment thread .github/workflows/database.yml
Comment thread .github/workflows/database.yml
Comment thread .github/workflows/database.yml Outdated
Comment thread .github/workflows/docker-k8s.yml Outdated
Comment thread Resources/docker/app/Dockerfile
Comment thread docker-compose.yml Outdated
Comment thread pyproject.toml Outdated
@FallenDeity

FallenDeity commented Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

was on vacation last week 🏖️ so changes/followup were bit delayed

@FallenDeity FallenDeity requested a review from Naramsim July 2, 2026 21:02
@Naramsim Naramsim merged commit 1e1600b into PokeAPI:master Jul 4, 2026
19 checks passed
@Naramsim

Naramsim commented Jul 4, 2026

Copy link
Copy Markdown
Member

Great! @FallenDeity your contributions are helping a lot! Thanks also to @phalt that submitted part of the work in another PR.

@pokeapi-machine-user

pokeapi-machine-user commented Jul 4, 2026

Copy link
Copy Markdown

A PokeAPI/api-data refresh has started. In ~45 minutes the staging branch of PokeAPI/api-data will be pushed with the new generated data.

The staging branch will be deployed in our staging environment and the entire API will be ready to review.

A Pull Request (master<-staging) will be also created at PokeAPI/api-data and assigned to the PokeAPI Core team to be reviewed. If approved and merged new data will soon be available worldwide at pokeapi.co.

pokeapi-machine-user added a commit to PokeAPI/api-data that referenced this pull request Jul 4, 2026
@pokeapi-machine-user
[Updater Bot] Regenerate data from PokeAPI/pokeapi#1553
b9149d4
@pokeapi-machine-user pokeapi-machine-user mentioned this pull request Jul 4, 2026
Closed
@pokeapi-machine-user

pokeapi-machine-user commented Jul 4, 2026

Copy link
Copy Markdown

The updater script has finished its job and has now opened a Pull Request towards PokeAPI/api-data with the updated data.

The Pull Request can be seen deployed in our staging environment when CircleCI deploy will be finished (check the start time of the last build).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phalt phalt phalt left review comments
@Naramsim Naramsim Naramsim approved these changes
+1 more reviewer
@jemarq04 jemarq04 jemarq04 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@FallenDeity @Naramsim @pokeapi-machine-user @phalt @jemarq04